Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add in VS 2022 support. #5956

Merged
merged 2 commits into from Oct 5, 2021
Merged

Add in VS 2022 support. #5956

merged 2 commits into from Oct 5, 2021

Conversation

michaelgsharp
Copy link
Member

Added in support for VS 2022 in the native build.

Also added documentation stating the version of CMake needed for 2022.

@codecov
Copy link

codecov bot commented Oct 5, 2021

Codecov Report

Merging #5956 (a4d6ad7) into main (6ae2cff) will increase coverage by 0.04%.
The diff coverage is 78.63%.

@@            Coverage Diff             @@
##             main    #5956      +/-   ##
==========================================
+ Coverage   68.19%   68.23%   +0.04%     
==========================================
  Files        1142     1142              
  Lines      242367   242803     +436     
  Branches    25355    25385      +30     
==========================================
+ Hits       165287   165686     +399     
- Misses      70400    70421      +21     
- Partials     6680     6696      +16     
Flag Coverage Δ
Debug 68.23% <78.63%> (+0.04%) ⬆️
production 62.89% <78.63%> (+0.01%) ⬆️
test 88.61% <ø> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...rosoft.Data.Analysis/ArrowStringDataFrameColumn.cs 63.54% <0.00%> (-0.47%) ⬇️
...crosoft.Data.Analysis/DataFrame.BinaryOperators.cs 19.64% <ø> (ø)
src/Microsoft.Data.Analysis/DataFrame.cs 86.68% <ø> (ø)
...t.Data.Analysis/DataFrameColumn.BinaryOperators.cs 5.55% <ø> (ø)
...lysis/PrimitiveDataFrameColumn.BinaryOperations.cs 40.30% <ø> (ø)
...imitiveDataFrameColumn.ReversedBinaryOperations.cs 50.00% <ø> (ø)
...ata.Analysis/PrimitiveDataFrameColumnArithmetic.cs 49.49% <ø> (ø)
...Analysis/StringDataFrameColumn.BinaryOperations.cs 79.41% <ø> (ø)
src/Microsoft.Data.Analysis/Strings.Designer.cs 43.26% <0.00%> (-0.95%) ⬇️
src/Microsoft.Extensions.ML/PoolLoader.cs 65.00% <ø> (ø)
... and 268 more

src/Native/build.cmd Outdated Show resolved Hide resolved
Copy link
Member

@tarekgh tarekgh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a little comment. LGTM otherwise.

@michaelgsharp michaelgsharp merged commit 54a3cb9 into dotnet:main Oct 5, 2021
@michaelgsharp michaelgsharp deleted the vs2022 branch October 5, 2021 18:02
Copy link
Member

@safern safern left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we have any reason to keep support for VS2015 and VS2017 around?

@michaelgsharp
Copy link
Member Author

This is just for building ML.NET. Do you think there is a need to remove the ability to build it on older versions of visual studio? I'm not sure if people are still using older versions or not. Is there a way to see?

@safern
Copy link
Member

safern commented Oct 8, 2021

There is not, it is not a big deal, we just in dotnet/runtime remove support for the old ones where the TFMs we use are not even supported.

@ghost ghost locked as resolved and limited conversation to collaborators Mar 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants